Skip to content

feat: add daemon cleanup stress test and tighten stream foreground handling#59

Merged
DjDeveloperr merged 2 commits into
mainfrom
add-cli-cleanup-stress-test
May 23, 2026
Merged

feat: add daemon cleanup stress test and tighten stream foreground handling#59
DjDeveloperr merged 2 commits into
mainfrom
add-cli-cleanup-stress-test

Conversation

@DjDeveloperr
Copy link
Copy Markdown
Collaborator

Summary

  • Added a dedicated daemon cleanup stress harness that starts isolated temporary project daemons, exercises health/metrics, and verifies process-group, listener-port, and metadata cleanup.
  • Wired the harness into npm run test:stress:daemon and documented it under testing.
  • Tightened viewer foreground detection so hidden or offscreen canvases stop reporting as active, reducing unnecessary encoder work.
  • Paired that client signal with native session cleanup so the video path pauses frame input when the last subscriber leaves and resumes when streaming restarts.
  • Documented the current encoding behavior and the low-footprint interpretation of viewer activity.

Testing

  • cargo test --manifest-path server/Cargo.toml
  • cargo fmt --manifest-path server/Cargo.toml --check
  • npm run --prefix client typecheck
  • npx prettier --check ...
  • node --check scripts/stress/daemon-cleanup.mjs
  • Ran scripts/stress/daemon-cleanup.mjs once against the debug binary with a passing cleanup result

@DjDeveloperr DjDeveloperr marked this pull request as ready for review May 21, 2026 18:35
@DjDeveloperr DjDeveloperr force-pushed the add-cli-cleanup-stress-test branch from 0c8b017 to da8f364 Compare May 22, 2026 21:39
@DjDeveloperr DjDeveloperr merged commit 053e81b into main May 23, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant